Skip to content

Conversation

johabort
Copy link
Member

Gjennomgang av tilgjengeligheten i appen, med vekt på semantisk HTML og
tastaturnavigasjon.

  • Klikkbare <div>-er byttet til <button> der elementet faktisk utfører
    en handling: filmkort, pilene og midtsonen i hero-karusellen
  • MovieCard er nå <article>, og tittelen ligger i en <button> som
    dekker hele kortet via ::after
  • Felles fokusmarkering via én :focus-visible-regel i index.css, i
    stedet for duplisert regel i fem filer
  • Fokus flyttes inn i dialogen ved åpning og tilbake til filmkortet ved
    lukking
  • Fokusfelle: Tab og Shift+Tab sykler mellom knappene i dialogen i stedet
    for å vandre ut i siden bak
  • Escape lukker dialogen
  • Scroll-lås på <body> mens dialogen er åpen
  • Snapshot for MovieCard oppdatert, da DOM-strukturen er endret
  • Manuelt testet med tastatur: tab-rekkefølge, fokusmarkering, Escape og
    fokusfelle i dialogen

Fixes #40

@johabort johabort requested a review from a team as a code owner September 17, 2026 16:28
Copy link
Member

@albertpl albertpl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bra knapper

@albertpl albertpl merged commit ef744eb into main Sep 18, 2026
@clemenbe clemenbe deleted the quickfix branch September 18, 2026 08:35
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disable scrolling i detailed view
2 participants